Skip to content

[UR][Offload] Update to use ol_symbol_handle_t #19389

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 15, 2025

Conversation

RossBrunton
Copy link
Contributor

@RossBrunton RossBrunton commented Jul 10, 2025

Recent liboffload changes have changed this interface a little

@RossBrunton RossBrunton requested a review from a team as a code owner July 10, 2025 14:14
@RossBrunton RossBrunton requested a review from callumfare July 10, 2025 14:14
Recent liboffload changes have changed this interface a little
@RossBrunton RossBrunton changed the title [UR][Offload] Rename ol_kernel_handle_t -> ol_symbol_handle_t [UR][Offload] Update to use ol_symbol_handle_t Jul 14, 2025
@RossBrunton RossBrunton requested a review from callumfare July 14, 2025 09:07
@RossBrunton
Copy link
Contributor Author

RossBrunton commented Jul 15, 2025

@intel/llvm-gatekeepers Please merge.

The offload build is failing because the version of liboffload it uses is out of date and isn't being updated. It's going to be disabled in #19445 . I've verified that offload builds locally.

EDIT: Wrong link, sorry!

@sarnex sarnex merged commit ede5e44 into intel:sycl Jul 15, 2025
45 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants